Skip to content

Remove guardrails ack flag and de-OpenAI README#5

Merged
toodimes merged 5 commits intomainfrom
davidastor/remove-danger-ack
Mar 5, 2026
Merged

Remove guardrails ack flag and de-OpenAI README#5
toodimes merged 5 commits intomainfrom
davidastor/remove-danger-ack

Conversation

@toodimes
Copy link
Copy Markdown
Owner

@toodimes toodimes commented Mar 5, 2026

Context

Removes unnecessary friction from Symphony startup and makes the codebase less OpenAI-specific.

TL;DR

Remove the verbose --i-understand-that-this-will-be-running-without-the-usual-guardrails CLI flag and strip OpenAI references from the README.

Summary

  • Removed the guardrails acknowledgement requirement that blocked startup without a verbose flag
  • Deleted the require_guardrails_acknowledgement/1 and acknowledgement_banner/0 functions
  • Updated CLI tests to remove the ack-flag test and simplify remaining tests
  • Replaced OpenAI-specific URLs with generic placeholders in README (Codex App Server docs link, Harness engineering link, GitHub clone URL, and model references)

Alternatives

  • Could have kept the ack flag as optional, but since it's universal friction with no benefit, removal is cleaner

Test Plan

  • mix test test/symphony_elixir/cli_test.exs — all 6 tests pass
  • Verified no other tests reference the ack flag

toodimes added 5 commits March 5, 2026 13:23
The verbose --i-understand-that-this-will-be-running-without-the-usual-guardrails
CLI flag added unnecessary friction to starting Symphony. Remove it along with the
banner and associated test.

Also strip OpenAI-specific URLs and model references from the README, replacing
them with generic placeholders.
The project lives at the repo root now, so the cd elixir prefixes in
hooks are wrong. Also points the clone URL at toodimes/symphony.
The prompt had massive redundancy — the same rules were restated across
Default posture, Status map, Step 0, Step 1, Step 2, Guardrails, and
Completion bar sections. Merged overlapping sections, eliminated
duplicate instructions, and condensed 23 execution sub-steps into 9.
The in-repo WORKFLOW.md now uses team_key instead of project_slug,
string-form active_states, updated clone URL and hook paths.
@toodimes toodimes merged commit daeb1c5 into main Mar 5, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant